Skip to content

Fix fuzzer issue template#6963

Merged
connortsui20 merged 2 commits intodevelopfrom
ct/fuzz-template-cmd
Mar 16, 2026
Merged

Fix fuzzer issue template#6963
connortsui20 merged 2 commits intodevelopfrom
ct/fuzz-template-cmd

Conversation

@connortsui20
Copy link
Contributor

Summary

From #6936 I do not understand why we are so against being descriptive in these templates given most of what claude says is complete nonsense. If we want concision, the reproduction steps still need to make sense.

This changes the fuzzer template to have 2 different details sections, one with a full and correct description of how to reproduce, and the other with a single command for people who know what they are doing. The maintainers do not need to look at the first section.

Testing

N/A

Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
@connortsui20 connortsui20 added the changelog/chore A trivial change label Mar 13, 2026
@connortsui20 connortsui20 changed the title fix fuzzer issue template Fix fuzzer issue template Mar 13, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Mar 13, 2026

Merging this PR will improve performance by 18.36%

⚡ 2 improved benchmarks
✅ 1007 untouched benchmarks
⏩ 1515 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation bitwise_not_vortex_buffer_mut[1024] 535.6 ns 477.2 ns +12.22%
Simulation bitwise_not_vortex_buffer_mut[128] 376.1 ns 317.8 ns +18.36%

Comparing ct/fuzz-template-cmd (2239159) with develop (01669a0)

Open in CodSpeed

Footnotes

  1. 1515 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@joseph-isaacs
Copy link
Contributor

We should optimize for small issue descriptions and someone familiar with the fuzzer.
We could have a hidden getting started section on the bottom, but that should be the less easy thing to access.

@joseph-isaacs
Copy link
Contributor

I am very happy to hide more of the issue, I just really don't want to scroll past instructions I already know each time I open a fuzzer issue

@connortsui20
Copy link
Contributor Author

In that case can we hide all of the Claude output? It's a huge wall of text that in my experience has rarely been useful.

And if we want to optimize for readability and convenience we should hide the debug output completely as well because that also usually tells us nothing more than the backtrace already gives us.

I can keep going, but to me this conversation makes zero sense when we can just collapse everything into details. The current template is already small enough that you don't need to scroll to anything.

Copy link
Contributor

@joseph-isaacs joseph-isaacs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG, I am bad at reading diff markdown

@connortsui20 connortsui20 enabled auto-merge (squash) March 16, 2026 13:10
@connortsui20 connortsui20 merged commit 34bd5ba into develop Mar 16, 2026
52 checks passed
@connortsui20 connortsui20 deleted the ct/fuzz-template-cmd branch March 16, 2026 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants